Skip to content

fix(acp): title Codex/ACP sessions with Buzz agent display name#2372

Open
Bartok9 wants to merge 1 commit into
block:mainfrom
Bartok9:bartok9/acp-session-title-display-name
Open

fix(acp): title Codex/ACP sessions with Buzz agent display name#2372
Bartok9 wants to merge 1 commit into
block:mainfrom
Bartok9:bartok9/acp-session-title-display-name

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 22, 2026

Copy link
Copy Markdown

Summary

Codex (and similar ACP harness UIs) title sessions from the start of the injected prompt. Every managed Buzz agent shared the same [Base] You are operating inside the Buzz platform… opener, so session lists showed identical truncated base text instead of the real agent name (Fizz, Varys, …).

This wires the managed agent display name from Desktop → buzz-acp and puts a short labeled [Session] section first in:

  • session/new systemPrompt framing (protocol v2 / modern path)
  • legacy user-message framing (prepend_base_for_legacy + format_prompt sections)

Base-prompt substance is unchanged. Desktop observers that split on [Base] / [System] / [Workspace] keep working — [Session] is an additive leading section.

Closes #2334

Changes

  • Desktop (managed_agents/runtime.rs): set BUZZ_ACP_DISPLAY_NAME from display_name, else name
  • buzz-acp config: --agent-display-name / BUZZ_ACP_DISPLAY_NAME
  • pool: framed_system_prompt + legacy prepend put [Session]\n{name} first; sanitize controls / collapse whitespace / cap 80 chars
  • queue: legacy format_prompt injects [Session] before [Base] when present
  • Tests: title-before-base, title-before-workspace, sanitize, legacy prepend

Out of scope

Test plan

  • cargo test -p buzz-acp --lib session_title
  • cargo test -p buzz-acp --lib framed_system_prompt
  • cargo test -p buzz-acp --lib prepend_base
  • cargo test -p buzz-acp --lib pool::tests (96 ok)
. ./bin/activate-hermit
cargo test -p buzz-acp --lib session_title
cargo test -p buzz-acp --lib pool::tests

AI-assisted; human-reviewed.

@Bartok9
Bartok9 requested a review from a team as a code owner July 22, 2026 14:47
Codex and similar ACP UIs label sessions from the opening of systemPrompt /
the first user message. Managed Buzz agents all started with the shared
[Base] platform prefix, so session lists showed identical truncated base
text instead of Fizz/Varys/etc.

Desktop now passes BUZZ_ACP_DISPLAY_NAME (display_name, else name). buzz-acp
puts a short [Session] section first in session/new systemPrompt framing and
on the legacy user-message path, without rewriting base-prompt substance or
breaking [Base]/[System]/[Workspace] observer splits.

Closes block#2334

Signed-off-by: Bartok9 <danielrpike9@gmail.com>
@Bartok9
Bartok9 force-pushed the bartok9/acp-session-title-display-name branch from a967db1 to a5e7c14 Compare July 22, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Managed ACP sessions show base prompt as title in Codex (and other harness UIs)

1 participant